Skip to content

docs: update for PR #42#13

Open
github-actions[bot] wants to merge 1 commit intomainfrom
docs/pr-42
Open

docs: update for PR #42#13
github-actions[bot] wants to merge 1 commit intomainfrom
docs/pr-42

Conversation

@github-actions
Copy link
Contributor

Updates documentation for node-banana PR #42.

Changes Made

Changelog (pages/changelog.mdx)

  • Added entry under "Fixed" section for version 1.3.0
  • Documents fix for race condition causing duplicate workflow executions when rapidly triggering execution

Summary of PR #42

PR #42 fixed a race condition in workflow execution by moving the isRunning guard before async logger calls in both executeWorkflow() and regenerateNode() methods. This prevents concurrent calls from passing the guard during the async gap, which could cause duplicate workflow executions when users rapidly click the execute button.

The fix includes 4 new integration tests verifying:

  1. Concurrent executeWorkflow() calls only execute nodes once
  2. isRunning is set synchronously before any await
  3. Concurrent regenerateNode() calls only execute once
  4. Multiple disconnected nodes produce exactly the expected number of fetch calls

🤖 Generated with Claude Code

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
@vercel
Copy link

vercel bot commented Jan 31, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
node-banana-docs Ready Ready Preview, Comment Jan 31, 2026 11:30am

@coderabbitai
Copy link

coderabbitai bot commented Jan 31, 2026

Important

Review skipped

Bot user detected.

To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

  • 🔍 Trigger a full review

Comment @coderabbitai help to get the list of available commands and usage tips.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant